don't put plurals in names

I have renamed the module name

@Tisho99 pre-commit

@Tisho99 pre-commit

@ValentinVinagre I have fixed the precommit

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@Tisho99 Thanks for this. My question: isn't it a little bit to harsh to restrict copy of the whole sale order ? Maybe not copying those products in lines ?

Hello @rousseldenis, thank you for your interest.

We had considered allowing duplicate orders by omitting lines with archived products, but we ruled it out because users might duplicate orders and not realize that some lines had been deleted.

However, if the use case is important, we could add this option via company configuration, allowing duplicate orders and adding a note with the lines that could not be duplicated, for example in the chatter.

CC @ValentinVinagre

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Exactly what @Tisho99 pointed out. We ruled it out due to the high likelihood of user error. Another option was to indicate in an internal note the products not transferred to the new sale... but we saw the same likelihood of errors. So we opted for this sole method; but it could easily be expanded.

@rousseldenis Did you have the chance to check our comments?

THX.

Merging as we didn't receive any blocking feedback. This module can be improved in a future.

/ocabot merge nobump

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-3884-by-HaraldPanten-bump-nobump, awaiting test results.

Congratulations, your PR was merged at 7b6888d. Thanks a lot for contributing to OCA. ❤️